Skip to content

Fix Initial Focus#4

Open
thestr4ng3r wants to merge 1 commit intotwcapps:developfrom
thestr4ng3r:fix-initial-focus
Open

Fix Initial Focus#4
thestr4ng3r wants to merge 1 commit intotwcapps:developfrom
thestr4ng3r:fix-initial-focus

Conversation

@thestr4ng3r
Copy link
Copy Markdown

The initial/default focus didn't add the "focus" class to the element because $el of the FocusElement will be undefined in bind().

@thestr4ng3r
Copy link
Copy Markdown
Author

@bartsidee @twcapps ping

Copy link
Copy Markdown
Collaborator

@victortwc victortwc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, @thestr4ng3r! Is there any particular situation that you've ran into that lead to that change?

@victortwc victortwc changed the base branch from master to develop February 6, 2020 15:15
@thestr4ng3r
Copy link
Copy Markdown
Author

I don't remember anymore, the project where I used this has already been finished long ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants